Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix growpart error with older blkid versions #54

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

balcsida
Copy link
Contributor

@balcsida balcsida commented Mar 7, 2024

Certain older distributions (e.g. CentOS 7.9) include outdated versions (from 2013) of blkid.
Because long options were added in 2017, using the "original", short form to check the version would make this script work with these blkid versions instead of failing with blkid: invalid option -- '-'

@holmanb
Copy link
Member

holmanb commented Apr 1, 2024

Certain older distributions (e.g. CentOS 7.9) include outdated versions (from 2013) of blkid.

@balcsida Will the Centos 7 package pull in this change? I didn't think they were pulling in new software updates.

Please sign the cla.

@balcsida
Copy link
Contributor Author

balcsida commented Apr 5, 2024

Hey @holmanb,

That might be true that CentOS 7 (and Debian 9 "Stretch") will not recieve updates, growpart downloaded from here for my partition resizing needs. I understand that supporting older, unsupported versions might not be a priority, however, this really small change would make growpart compatible with older systems with blkid version lower than 2.30.

Thanks for the heads-up, I've signed the CLA.

Kind regards,
@balcsida

@holmanb
Copy link
Member

holmanb commented Apr 5, 2024

however, this really small change would make growpart compatible with older systems with blkid version lower than 2.30.

agreed, and no objections to the change

Thanks for the heads-up, I've signed the CLA.

Confirmed

@holmanb holmanb self-assigned this Apr 5, 2024
@holmanb
Copy link
Member

holmanb commented Apr 5, 2024

@balcsida Could you please rebase on the tip of main to pick up the ci fix? Cnce that passes I'll merge.

@balcsida balcsida force-pushed the fix/growpart-blkid-error branch from 6d01762 to 1e5c0fc Compare April 5, 2024 21:24
@holmanb holmanb added this pull request to the merge queue Apr 5, 2024
Merged via the queue into canonical:main with commit 49e5dd7 Apr 5, 2024
2 checks passed
MingcongBai pushed a commit to AOSC-Tracking/cloud-utils that referenced this pull request May 13, 2024
Certain older distributions (e.g. CentOS 7.9) include outdated versions
(from 2013) of `blkid`.
Because [long options were added in
2017](util-linux/util-linux@15a74f7),
using the "original", short form to check the version would make this
script work with these `blkid` versions instead of failing with `blkid:
invalid option -- '-'`
@balcsida balcsida deleted the fix/growpart-blkid-error branch October 1, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants